Skip to content

Improve error for users with no snowflake license#230

Merged
anisaoshafi merged 2 commits intomainfrom
drg-818-improve-sno-access-error-for-non-sno-users
May 7, 2026
Merged

Improve error for users with no snowflake license#230
anisaoshafi merged 2 commits intomainfrom
drg-818-improve-sno-access-error-for-non-sno-users

Conversation

@anisaoshafi
Copy link
Copy Markdown
Contributor

@anisaoshafi anisaoshafi commented May 6, 2026

Follow up from #217 (comment)
Improve Snowflake access error for users that don't have it in their licence, and guide them on how to continue.

Before After
Screenshot 2026-05-06 at 18 32 40 Screenshot 2026-05-06 at 19 09 57

@anisaoshafi anisaoshafi force-pushed the drg-818-improve-sno-access-error-for-non-sno-users branch from 78a112a to cf8d806 Compare May 6, 2026 17:17
@anisaoshafi anisaoshafi requested a review from gtsiolis May 6, 2026 17:18
if errors.As(err, &licErr) && c.EmulatorType == config.EmulatorSnowflake {
errCode = telemetry.ErrCodeLicenseInvalid
sink.Emit(output.ErrorEvent{
Title: "Your license does not include the Snowflake emulator.",
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

praise: Short and sweet.

Comment thread internal/container/start.go Outdated
@anisaoshafi anisaoshafi requested a review from gtsiolis May 7, 2026 11:20
@anisaoshafi anisaoshafi force-pushed the drg-818-improve-sno-access-error-for-non-sno-users branch from ff20986 to be3c25f Compare May 7, 2026 11:45
@anisaoshafi anisaoshafi merged commit 9fdb92f into main May 7, 2026
12 checks passed
@anisaoshafi anisaoshafi deleted the drg-818-improve-sno-access-error-for-non-sno-users branch May 7, 2026 12:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants